Skip to content

Introduce 'USE_COMPILER_DEFAULT_LIBC' cmake option for switching from jerry-libc to a libc, provided by compiler #331

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 8, 2015

Conversation

ruben-ayrapetyan
Copy link
Contributor

No description provided.

@ruben-ayrapetyan ruben-ayrapetyan added enhancement An improvement normal infrastructure Related to GH Actions or the tested targets labels Jul 7, 2015
@ruben-ayrapetyan ruben-ayrapetyan force-pushed the use-compiler-default-libc-cmake-option-dev branch from 238c4df to d6b9f2c Compare July 7, 2015 22:35
@ruben-ayrapetyan ruben-ayrapetyan force-pushed the use-compiler-default-libc-cmake-option-dev branch 2 times, most recently from 0baccab to ecc0e20 Compare July 7, 2015 23:38
@seanshpark
Copy link
Contributor

Good to me

@ruben-ayrapetyan ruben-ayrapetyan force-pushed the use-compiler-default-libc-cmake-option-dev branch from ecc0e20 to 192a85d Compare July 7, 2015 23:40
@egavrin
Copy link
Contributor

egavrin commented Jul 8, 2015

Good to me

@egavrin egavrin assigned LaszloLango and unassigned egavrin Jul 8, 2015
$(error EXTERNAL_LIBC_INTERFACE should not be specified in case compiler's default libc is used)
endif
else
EXTERNAL_LIBC_INTERFACE ?= $(shell pwd)/third-party/nuttx/include
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think after #330 this isn't needed anymore.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, seems so. I'll rebase to current master.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@ruben-ayrapetyan ruben-ayrapetyan force-pushed the use-compiler-default-libc-cmake-option-dev branch from 192a85d to 718c69f Compare July 8, 2015 11:37
@LaszloLango
Copy link
Contributor

lgtm

@egavrin
Copy link
Contributor

egavrin commented Jul 8, 2015

make push

…o a libc, provided by compiler.

JerryScript-DCO-1.0-Signed-off-by: Ruben Ayrapetyan [email protected]
@egavrin egavrin force-pushed the use-compiler-default-libc-cmake-option-dev branch from 718c69f to 393d693 Compare July 8, 2015 12:02
@egavrin egavrin merged commit 393d693 into master Jul 8, 2015
@egavrin egavrin deleted the use-compiler-default-libc-cmake-option-dev branch July 8, 2015 15:35
seanshpark added a commit to seanshpark/jerryscript that referenced this pull request Jul 9, 2015
seanshpark added a commit to seanshpark/jerryscript that referenced this pull request Jul 12, 2015
LaszloLango pushed a commit to LaszloLango/jerryscript that referenced this pull request Jul 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement infrastructure Related to GH Actions or the tested targets normal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants